Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Solves issue #428 Linking error: SetPickingColor #430

Closed
wants to merge 7 commits into from

Conversation

lpgasparotto
Copy link
Contributor

This solves the linking errors which many users reported. This was reported on issue #428 Linking error: SetPickingColor.

This solves the linking errors which many users reported. This was reported on issue Heeks#428 Linking error: SetPickingColor.
Picking.cpp is required for HeeksCNC compilation but it's not currently installed by HeeksCAD.
I fixed wrong translations and replaced "y" with "&" in the menu titles translations which makes to show the icons correctly in the dropdown menus.
Copy link
Contributor Author

@lpgasparotto lpgasparotto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please dismiss this file

@neomilium
Copy link
Member

@lpgasparotto Your translation improvements have been extracted from this PR and merged into master: 5c9410e

Regarding the linking error, I reworked your job and moved the shared files (Picking.*) into the dedicated plugins interface directory (ie. interface/) in order to use it from HeeksCNC.

Thanks for your contribution!

@neomilium neomilium closed this Jun 13, 2020
@lpgasparotto
Copy link
Contributor Author

You're welcome!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants